Skip to content

[stable33] fix(settings): display birthday correctly regardless of browsers timezone#61714

Merged
AndyScherzinger merged 2 commits into
stable33from
backport/61586/stable33
Jul 8, 2026
Merged

[stable33] fix(settings): display birthday correctly regardless of browsers timezone#61714
AndyScherzinger merged 2 commits into
stable33from
backport/61586/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 2, 2026

Copy link
Copy Markdown

Backport of #61586

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot Bot requested review from odzhychko, skjnldsv and susnux July 2, 2026 07:46
@backportbot backportbot Bot added bug 3. to review Waiting for reviews labels Jul 2, 2026
@backportbot backportbot Bot added this to the Nextcloud 33.0.7 milestone Jul 2, 2026
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from b18780f to 2c94283 Compare July 3, 2026 07:57
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from 2c94283 to 2df9a5a Compare July 3, 2026 08:28
odzhychko added 2 commits July 6, 2026 03:20
`savePrimaryAccountProperty` does only allow  `string | boolean`.
Incidentally, relying on `@nextcloud/axios` default serialization, `Date`s where serialized using `toISOString`.
See https://github.com/axios/axios/blob/a7d7a714498452e611943d9d8e5c2ed8b00aa6b9/lib/helpers/toFormData.js#L134

This change corrects the usage of `savePrimaryAccountProperty` and makes serialization explicit.

Signed-off-by: Oleksandr Dzhychko <hey@oleks.dev>
…zone

This also avoid shifting dates after user input.

Resolves #49828

Signed-off-by: Oleksandr Dzhychko <hey@oleks.dev>
@odzhychko odzhychko force-pushed the backport/61586/stable33 branch from 2df9a5a to 31fcb53 Compare July 6, 2026 03:24
@odzhychko

Copy link
Copy Markdown
Contributor

Ready for review. I'm fairly sure, the failing Cypress tests are not related to the changes in this PR.

@odzhychko odzhychko marked this pull request as ready for review July 6, 2026 03:55
@odzhychko odzhychko requested a review from a team as a code owner July 6, 2026 03:55
@odzhychko odzhychko requested review from sorbaugh and removed request for a team July 6, 2026 03:55
@odzhychko

Copy link
Copy Markdown
Contributor

/compile rebase

@odzhychko

Copy link
Copy Markdown
Contributor

Ready for review. I'm fairly sure, the failing Cypress tests are not related to the changes in this PR.

@ChristophWurst Could you please force merge this PR?

@odzhychko odzhychko added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 8, 2026
@AndyScherzinger AndyScherzinger merged commit 3b31202 into stable33 Jul 8, 2026
125 of 128 checks passed
@AndyScherzinger AndyScherzinger deleted the backport/61586/stable33 branch July 8, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants